Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PMM-11079 Remove Integrated Alerting #2309

Merged
merged 31 commits into from
Oct 12, 2023
Merged

Conversation

artemgavrilov
Copy link
Contributor

@artemgavrilov artemgavrilov commented Jun 26, 2023

PMM-11079

Link to the Feature Build: SUBMODULES-3324

If this PR adds or removes or alters one or more API endpoints, please review and add or update the relevant API documents as well:

  • API Docs updated

If this PR is related to some other PRs in this or other repositories, please provide links to those PRs:

  • Links to related pull requests (optional).

@codecov
Copy link

codecov bot commented Jun 26, 2023

Codecov Report

Merging #2309 (b5b695f) into v3 (dfdbc87) will increase coverage by 0.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##               v3    #2309      +/-   ##
==========================================
+ Coverage   43.97%   44.09%   +0.12%     
==========================================
  Files         362      351      -11     
  Lines       43252    41136    -2116     
==========================================
- Hits        19018    18139     -879     
+ Misses      22586    21434    -1152     
+ Partials     1648     1563      -85     
Flag Coverage Δ
admin 10.41% <ø> (ø)
agent 52.87% <ø> (-0.15%) ⬇️
vmproxy 68.57% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
managed/cmd/pmm-managed/main.go 0.00% <ø> (ø)
managed/models/database.go 42.91% <ø> (ø)
managed/models/models.go 81.37% <ø> (+4.06%) ⬆️
managed/models/settings.go 100.00% <ø> (ø)
managed/models/settings_helpers.go 63.95% <ø> (-5.54%) ⬇️
managed/models/template_model.go 77.27% <ø> (-15.32%) ⬇️
managed/models/template_model_reform.go 77.27% <ø> (ø)
managed/services/alertmanager/alertmanager.go 26.34% <ø> (-24.73%) ⬇️
managed/services/grafana/client.go 42.38% <ø> (-2.21%) ⬇️
managed/services/management/alerting/service.go 21.47% <ø> (-1.22%) ⬇️
... and 4 more

... and 6 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@artemgavrilov artemgavrilov marked this pull request as ready for review June 30, 2023 17:18
@artemgavrilov artemgavrilov requested review from a team, BupycHuk and ademidoff as code owners June 30, 2023 17:18
@artemgavrilov artemgavrilov requested review from idoqo and removed request for a team June 30, 2023 17:18
@ademidoff
Copy link
Member

ademidoff commented Jun 30, 2023

I believe there could be more mentions of Integrated Alerting in the code, for example

// ErrAlertingDisabled means Integrated Alerting is disabled and IA APIs can't be executed.

I suggest we update them all 👍

managed/services/server/deps.go Outdated Show resolved Hide resolved
@artemgavrilov
Copy link
Contributor Author

artemgavrilov commented Jul 3, 2023

I removed api/swagger/swagger-dev-only.json because there is no target that updates it anymore.

@artemgavrilov
Copy link
Contributor Author

This PR is postponed until PMM v3

@artemgavrilov artemgavrilov changed the base branch from main to v3 September 28, 2023 09:48
@artemgavrilov
Copy link
Contributor Author

artemgavrilov commented Oct 12, 2023

@BupycHuk @ademidoff I think it's ready to be merged. Tests failure is unrelated to these changes.

@BupycHuk BupycHuk merged commit b108991 into v3 Oct 12, 2023
31 of 32 checks passed
@BupycHuk BupycHuk deleted the PMM-11079-remove-integrated-alerting branch October 12, 2023 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants